Skip to content

[3.11] gh-110819: Fix ‘kind’ may be used uninitialized warning in longobject (GH-116599)#116650

Merged
sobolevn merged 1 commit into
python:3.11from
sobolevn:backport-eb947cd-3.11
Mar 12, 2024
Merged

[3.11] gh-110819: Fix ‘kind’ may be used uninitialized warning in longobject (GH-116599)#116650
sobolevn merged 1 commit into
python:3.11from
sobolevn:backport-eb947cd-3.11

Conversation

@sobolevn

@sobolevn sobolevn commented Mar 12, 2024

Copy link
Copy Markdown
Member

(cherry picked from commit eb947cd)

Note the change: on 3.11 kind is defined as enum PyUnicode_Kind: https://github.com/python/cpython/blob/3.11/Objects/longobject.c#L1938C5-L1938C24 So, I've changed int kind to be enum PyUnicode_Kind as it used to be.

@sobolevn

Copy link
Copy Markdown
Member Author

This did not go well, there are extra files. I will recreate the PR. Sorry, this is cherry_picker doing strange things 🤔

…in `longobject` (pythonGH-116599)

(cherry picked from commit eb947cd)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@sobolevn sobolevn force-pushed the backport-eb947cd-3.11 branch from d6db610 to 3651286 Compare March 12, 2024 11:05
@sobolevn

sobolevn commented Mar 12, 2024

Copy link
Copy Markdown
Member Author

Here we go, only a single file is left changed 👍

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I created #116665 to fix warnings on Modules/_ssl.c.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sobolevn sobolevn merged commit f3a2b62 into python:3.11 Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants